Skip to content

Signuperror : sign up toast error solved #338 done#345

Open
VinayLodhi1712 wants to merge 3 commits into
Bitbox-Connect:mainfrom
VinayLodhi1712:signuperror
Open

Signuperror : sign up toast error solved #338 done#345
VinayLodhi1712 wants to merge 3 commits into
Bitbox-Connect:mainfrom
VinayLodhi1712:signuperror

Conversation

@VinayLodhi1712

Copy link
Copy Markdown
Contributor

Title and Issue number

Title: Signuperror : sign up toast error solved #338 done

Issue No.:

Close #338


Screenshots/Video (mandatory)

20241102152002.mp4

Checklist:

  • [ X] I have mentioned the issue number in my Pull Request.
  • [X ] I have commented my code, particularly in hard-to-understand areas.
  • [ X] I have created a helpful and easy-to-understand README.md if it's a new page/tech stack.
  • [ X] I have gone through the contributing.md file before contributing.

Additional context (Mandatory):

Are you contributing under any Open-source programme?

  • [x ] I'm a GSSOC-EXT contributor
  • [ X] I'm a HACKTOBERFEST contributor

@netlify

netlify Bot commented Nov 2, 2024

Copy link
Copy Markdown

Deploy Preview for bitbox-in failed. Why did it fail? →

Name Link
🔨 Latest commit 9a204a4
🔍 Latest deploy log https://app.netlify.com/projects/bitbox-in/deploys/6725fa0d706b9c0007090b8f

@github-actions

github-actions Bot commented Nov 2, 2024

Copy link
Copy Markdown

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our chaotic CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

@Anuj3553

Anuj3553 commented Nov 2, 2024

Copy link
Copy Markdown
Collaborator

Here is the issue I don't want to login directly thats why i give the error response. Only if the user verification is complete by email then it can login through login

@Anuj3553

Anuj3553 commented Nov 2, 2024

Copy link
Copy Markdown
Collaborator

If you can figured out the way then it is very good

@VinayLodhi1712

Copy link
Copy Markdown
Contributor Author

can we redirect user to the login page..? He has to put the credentials in the input box then only he can login.

@Anuj3553

Anuj3553 commented Nov 4, 2024

Copy link
Copy Markdown
Collaborator

can we redirect user to the login page..? He has to put the credentials in the input box then only he can login.

Yes you can but here is some suggestion that you can modify the alert in frontend not on the backend side.

@VinayLodhi1712

VinayLodhi1712 commented Nov 4, 2024

Copy link
Copy Markdown
Contributor Author

What suggestions you want i have to keep in mind @Anuj3553 ...?

@Anuj3553

Anuj3553 commented Nov 5, 2024

Copy link
Copy Markdown
Collaborator

What suggestions you want i have to keep in mind @Anuj3553 ...?

You can navigate to login page but keep in mind we have to verify first from mail and only then we can able to login.

@VinayLodhi1712

Copy link
Copy Markdown
Contributor Author

What suggestions you want i have to keep in mind @Anuj3553 ...?

You can navigate to login page but keep in mind we have to verify first from mail and only then we can able to login.

then we have to check in backend with creating a variable in models of user whether user is verified or not. Because i think currently also without verifying also login will be done with id and password.

@Anuj3553

Anuj3553 commented Nov 6, 2024

Copy link
Copy Markdown
Collaborator

What suggestions you want i have to keep in mind @Anuj3553 ...?

You can navigate to login page but keep in mind we have to verify first from mail and only then we can able to login.

then we have to check in backend with creating a variable in models of user whether user is verified or not. Because i think currently also without verifying also login will be done with id and password.

Yes, its correct

@VinayLodhi1712

Copy link
Copy Markdown
Contributor Author

ok i will try to do it and when it done then i will create a pr but since now i have to work on the backend side and also checking the email and verification and then send message to frontend so i need time and also it becomes an level 3 issue so if i can do it successfully the its request that you plz increase the level of it.

@Anuj3553

Anuj3553 commented Nov 6, 2024

Copy link
Copy Markdown
Collaborator

ok i will try to do it and when it done then i will create a pr but since now i have to work on the backend side and also checking the email and verification and then send message to frontend so i need time and also it becomes an level 3 issue so if i can do it successfully the its request that you plz increase the level of it.

Definately i will increase your level from current to level3 ✌️.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: while signup successfully the message it is sending is correct but the red colored failure message it is showing.

2 participants